Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/main/ipc/ipc-schemas.test.ts`:
- Around line 130-141: The test for invalid windowBackgroundBlurRadius only
checks a fractional value and an over-max value; add a lower-bound failure case
by asserting schema.safeParse([{ windowBackgroundBlurRadius:
WINDOW_BACKGROUND_BLUR_MIN_RADIUS - 1 }]).success.toBe(false) (use the min
constant paired with WINDOW_BACKGROUND_BLUR_MAX_RADIUS) so the suite covers both
boundary sides; add this expectation alongside the existing safeParse checks in
the same it('rejects an invalid windowBackgroundBlurRadius value', ...) block
referencing schema.safeParse.

In `@src/main/utils/windowBackgroundBlur.test.ts`:
- Around line 12-35: Add unit tests for applyWindowBackgroundBlur that cover
both branches when the BrowserWindow API supports setBackgroundBlur and when it
does not: create minimal mock window objects (one with setBackgroundBlur and one
without) that spy on setBackgroundBlur and setBackgroundColor; call
applyWindowBackgroundBlur with values that exercise clamping (e.g., -12, 12,
WINDOW_BACKGROUND_BLUR_MAX_RADIUS) and with 0 vs non-zero to check background
color selection, then assert that when setBackgroundBlur exists it was called
with the clamped radius and when absent it was not called, and in both cases
assert setBackgroundColor was called with MAIN_WINDOW_OPAQUE_BACKGROUND for
radius 0 and MAIN_WINDOW_BLURRED_BACKGROUND for non-zero.

In `@src/renderer/settings/sections/Appearance.tsx`:
- Around line 52-53: The onChange handler (handleInputChange) currently calls
updateSettings on every slider event causing IPC/save/broadcast storms; change
it so onChange only updates local UI state, and debounce or delay calls to
updateSettings (or call updateSettings only on commit/end interaction) to limit
persistence frequency; implement this by adding a debounced wrapper (e.g.,
useRef + setTimeout or lodash.debounce) around updateSettings or adding an
onChangeCommitted/onMouseUp handler that calls updateSettings once, and ensure
functions named handleInputChange and updateSettings are the only places
modified.

In `@src/renderer/src/components/skills/FileContent.browser.test.tsx`:
- Around line 136-140: The test is scrolling the text node instead of the actual
scroll container; locate the nearest <pre> from the element returned by
screen.getByText(/wide command/) (use .closest('pre')), call scrollTo on that
pre element (not the text node), and update the assertion to verify the pre's
scrollLeft reflects the scroll (e.g., not 0 or equals the expected value) so the
horizontal scroll behavior is correctly tested.

In `@src/renderer/src/styles/globals.css`:
- Around line 131-132: Add a blank line before the "background: transparent;"
declaration to satisfy the stylelint rule declaration-empty-line-before; locate
the block containing "@apply text-foreground font-sans antialiased;" and insert
a single empty line directly above the "background: transparent;" line so the
"background" declaration is separated by one empty line from the previous
declaration.

---

Outside diff comments:
In `@src/main/services/settings.test.ts`:
- Around line 27-34: Add a dedicated test case to verify that areSettingsEqual
detects changes to windowBackgroundBlurRadius: create a new it(...) similar to
the existing primitive-diff test that calls areSettingsEqual(baseSettings, {
...baseSettings, windowBackgroundBlurRadius:
baseSettings.windowBackgroundBlurRadius + 1 }) and assert toBe(false). Reference
areSettingsEqual and baseSettings so the test fails if the blur comparison is
missing.

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/renderer/settings/sections/Appearance.tsx`:
- Around line 111-113: The effect that syncs blur draft with
windowBackgroundBlurRadius must cancel any pending debounce timer so a stale
pending update doesn't overwrite a broadcasted value; modify the component to
store the pending timer ID (e.g., a ref like blurRadiusTimerRef used by the
slider update logic) and call clearTimeout on that ref inside the
useComponentEffect that runs setBlurRadiusDraft(windowBackgroundBlurRadius), and
also clear/replace the timer whenever the slider starts a new debounce; ensure
the code paths around the slider handler that call updateSettings({
windowBackgroundBlurRadius: ... }) set the ref so it can be cleared by
useComponentEffect to prevent stale updates.